Skip to content

Breaking: Uses Swift Concurrency #152

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

NeedleInAJayStack
Copy link
Member

This removes the NIO dependency. It is breaking because it removes all Swift NIO-isms that were present in the public APIs (like EventLoopFuture and EventLoopGroup argument/return types).

Do not merge until GraphQLSwift/GraphQL#166 has been merged and tagged

paulofaria
paulofaria previously approved these changes Jun 24, 2025
@NeedleInAJayStack NeedleInAJayStack self-assigned this Jun 24, 2025
@NeedleInAJayStack NeedleInAJayStack force-pushed the feat/swift-concurrency branch 4 times, most recently from a561b89 to 48b6891 Compare July 15, 2025 05:42
@NeedleInAJayStack NeedleInAJayStack changed the title Draft: feat!: Uses swift concurrency under the hood Draft: Breaking: Uses Swift Concurrency Jul 21, 2025
@NeedleInAJayStack NeedleInAJayStack force-pushed the feat/swift-concurrency branch 3 times, most recently from 7292d74 to ef0ffea Compare August 11, 2025 05:42
This removes the NIO dependency. It is breaking because it removes all Swift NIO-isms that were present in the public APIs (like EventLoopFuture and EventLoopGroup argument/return types).
@NeedleInAJayStack NeedleInAJayStack changed the title Draft: Breaking: Uses Swift Concurrency Breaking: Uses Swift Concurrency Aug 11, 2025
@NeedleInAJayStack
Copy link
Member Author

@paulofaria This should be ready for review now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants